A new Pull Request was created by @stahlleiton for CMSSW_13_2_X.

It involves the following packages:

• RecoBTag/FeatureTools (reconstruction)
• RecoJets/JetProducers (reconstruction)

@mandrenguyen, @cmsbuild, @jfernan2 can you please review it and eventually sign? Thanks.
@gkasieczka, @jdolen, @emilbols, @rappoccio, @Ming-Yan, @seemasharmafnal, @Senphy, @AnnikaStein, @mariadalfonso, @jdamgov, @clelange, @missirol, @AlexDeMoor, @demuller, @JyothsnaKomaragiri, @yslai, @andrzejnovak, @nhanvtran, @schoef, @ahinzmann, @hqucms this is something you requested to watch as well.
@rappoccio, @sextonkennedy, @antoniovilela you are the release manager for this.

cms-bot commands are listed here

• Backported from Add use of unsubtracted jets in RecoBTag info producers #44422

cms-bot internal usage

Pull request #44475 was updated. @cmsbuild, @mandrenguyen, @jfernan2 can you please check and sign again.

@stahlleiton Was there a 13_3_X PR too?

@stahlleiton Was there a 13_3_X PR too?

#44579

please test

type btv

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-629736/38542/summary.html
COMMIT: 21c9119
CMSSW: CMSSW_13_2_X_2024-03-31-0000/el8_amd64_gcc11
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/44475/38542/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially added 1 lines to the logs
• Reco comparison results: 2 differences found in the comparisons
• DQMHistoTests: Total files compared: 48
• DQMHistoTests: Total histograms compared: 3198830
• DQMHistoTests: Total failures: 3
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3198805
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 47 files compared)
• Checked 207 log files, 159 edm output root files, 48 DQM output files
• TriggerResults: no differences found

Need to apply fix from #44591.

Pull request #44475 was updated. @jfernan2, @mmusich, @mandrenguyen, @Martin-Grunewald, @cmsbuild can you please check and sign again.

@stahlleiton it is possible to avoid the "merge" in the commit history of this PR?

backport of #44422

@stahlleiton it is possible to avoid the "merge" in the commit history of this PR?

The release I was working on was deleted. So I had to start with a new IB release. I can try to squash the commits if you prefer.

I can try to squash the commits if you prefer.

yes, rebasing against the latest 13.2.X IB (thus removing the merge) is the preferred option.

Pull request #44475 was updated. @Martin-Grunewald, @mandrenguyen, @cmsbuild, @mmusich, @jfernan2 can you please check and sign again.

I can try to squash the commits if you prefer.

yes, rebasing against the latest 13.2.X IB (thus removing the merge) is the preferred option.

done, I rebased against latest IB

done, I rebased against latest IB

thank you @stahlleiton

@cmsbuild, please test

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-629736/38564/summary.html
COMMIT: 96402f8
CMSSW: CMSSW_13_2_X_2024-03-31-0000/el8_amd64_gcc11
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/44475/38564/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially added 2 lines to the logs
• Reco comparison results: 6 differences found in the comparisons
• DQMHistoTests: Total files compared: 48
• DQMHistoTests: Total histograms compared: 3198830
• DQMHistoTests: Total failures: 3
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3198805
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 47 files compared)
• Checked 207 log files, 159 edm output root files, 48 DQM output files
• TriggerResults: no differences found

+hlt

• in this cycle the new parameter unsubjet_map of DeepBoostedJetTagInfoProducer has never been defined as untracked.

+1

This pull request is fully signed and it will be integrated in one of the next CMSSW_13_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_14_1_X is complete. This pull request will now be reviewed by the release team before it's merged. @antoniovilela, @rappoccio, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2)

+1